Skip to content

Махалин Николай#6

Open
makhalin wants to merge 19 commits intourfu-2016:masterfrom
makhalin:master
Open

Махалин Николай#6
makhalin wants to merge 19 commits intourfu-2016:masterfrom
makhalin:master

Conversation

@makhalin
Copy link
Copy Markdown

No description provided.

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Пройдено тестов 14 из 19

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍏 Пройдено тестов 19 из 19

@honest-hrundel
Copy link
Copy Markdown

@Victoria-Vladimirova обрати внимание решено доп. задание

@mokhov
Copy link
Copy Markdown

mokhov commented Oct 27, 2016

Николай, отправь PR ещё раз

@honest-hrundel
Copy link
Copy Markdown

🍏 Пройдено тестов 19 из 19

@honest-hrundel
Copy link
Copy Markdown

@FrodoTheTrue обрати внимание решено доп. задание

@FrodoTheTrue
Copy link
Copy Markdown

почему билд сломан?

}

function convertTime(time, timezone) {
time = /(\d{2}):(\d{2})\+(\d+)/.exec(time);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DRY

function convertSchedule(schedule, timezone) {
var newSchedule = [];
for (var name in schedule) {
if (schedule.hasOwnProperty(name)) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем эта проверка?

var newSchedule = [];
for (var name in schedule) {
if (schedule.hasOwnProperty(name)) {
var persSchedule = schedule[name]
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

persSchedule не очень понятное название

function getRobberyTimes(fullSchedule, duration) {
var robberyTimes = [];
var iter = [];
var iterMax = [];
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

хитро сделано с хранением числа в массиве :)
iter, iterMax здесь не подходят, не понятно что итерируем и как

getIters(iter, iterMax, fullSchedule);

while (iter[0] < iterMax[0]) {
var minMax = getMinMax(fullSchedule, iter);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

странное название для функции

*/
exports.getAppropriateMoment = function (schedule, duration, workingHours) {
console.info(schedule, duration, workingHours);
exports.getAppropriateMoment = function sad(schedule, duration, workingHours) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sad ? 😢 😞 😿

@FrodoTheTrue
Copy link
Copy Markdown

🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants